Skip to content

feat: support YAML 1.1 schema#17

Merged
ObserverOfTime merged 2 commits intomasterfrom
yaml11
May 22, 2025
Merged

feat: support YAML 1.1 schema#17
ObserverOfTime merged 2 commits intomasterfrom
yaml11

Conversation

@ObserverOfTime
Copy link
Member

This schema has a handful of incompatibilities with the YAML 1.2 (core) schema, most notably an extra 'timestamp' type.
I expected tree-sitter to prune the timestamp_scalar node when not using this schema, but apparently it doesn't.

Schemas can be switched by running tree-sitter build/parse/test with CFLAGS=-DYAML_SCHEMA=core/json/legacy.

See nvim-treesitter/nvim-treesitter#7512 (comment).

@ObserverOfTime ObserverOfTime requested a review from amaanq January 13, 2025 11:21
@char0n
Copy link

char0n commented Jan 13, 2025

Hi @ObserverOfTime,

Just making sure that this change will not change the default YAML 1.2 core scheme for this library, right?

@ObserverOfTime
Copy link
Member Author

It will not.

@ObserverOfTime ObserverOfTime merged commit 8be0fb4 into master May 22, 2025
5 checks passed
@ObserverOfTime ObserverOfTime deleted the yaml11 branch May 22, 2025 05:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants